-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(contracts): upgrade solidity version to 0.8.20 #109
Conversation
Updated and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: [email protected] |
b2dc770
to
e729c37
Compare
d829835
to
7c939c8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM, why specifically to 0.8.15?
I am getting errors related to openzeppelin imports in remix when I am using version higher than this. Will try to solve the error and use version 0.8.20 which is the latest version |
1d6865d
to
7a936bd
Compare
7a936bd
to
faa50b7
Compare
faa50b7
to
5cc0c6f
Compare
fca8f2a
to
d032ea1
Compare
@sachushaji I have updated the version to 0.8.20 which is the latest version used in openzeppelin contracts as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks Akash
This PR upgrades the version of solidity to 0.8.20 for contracts, updates node version, modifies the naming for private keys used for contract deployment.
WIN-666